Is @ $ array ['mogelijk_missing_key'] een anti-patroon?

De @ -operator onderdrukt foutmeldingen, en het gebruik ervan stelt mogelijk uw code in voor andere fouten en onverwacht gedrag die moeilijk op te sporen zijn. Het is dus zeker een antipatroon.
Daarom zou ik heel veel de voorkeur geven aan het tweede bit. Het maakt het veel duidelijker
dat het mogelijk niet aanwezig is in de array, en
wat de standaardwaarde is als deze niet aanwezig is
Om het beknopter te maken, kunt u de ternaire voorwaardelijke operator?: Gebruiken, zoals te zien is in het antwoord van Mark Baker. Iets minder code en meer symbolen, maar de betekenis wordt goed herkend.
|
Eigenlijk is de standaardvariatie het antipatroon. Als je gewoon isset ($ var)? $ Var: NULL gebruikt met de bedoeling om de "fout" te onderdrukken, dan heb je niets bereikt door de juiste syntaxis te gebruiken voor het onderdrukken van fouten. Het heeft hetzelfde resultaat, maar is minder leesbaar.
Mensen pleiten daarvoor vanwege de waargenomen "reinheid" en omdat het gebruik van isset een micro-optimalisatie is. Het vermijden van @ en het gebruik van isset als syntactische zoutvervanging is slechts programmering van de ladingcultus.
|
Of
$ value = (isset ($ array ['mogelijk_missing_key']))? $ array ['mogelijk_missing_key']: null;
|
Het negeren van waarschuwingen is beslist een antipatroon; dus ja, het is een antipatroon (en ik kan je garanderen dat als je leert om waarschuwingen te onderdrukken, een van hen terugkomt en je in het achterste bijt, zo niet erger).
Hoewel de tweede versie meer uitgebreid is, geeft het de niet-geïnitialiseerde variabele een bekende status (of kan het worden gebruikt om het probleem op te lossen, als de variabele zou moeten worden gevuld).
|
De derde optie:
$ value = (isset ($ array ['key'])? $ array ['key']: null);
Ik weet dat dit de vraag niet direct beantwoordt; Ik zou het als een opmerking hebben gezegd, maar het moest echt worden opgemaakt.
Het idee hier is dat als u probeert uw code korter te maken door een one-liner te gebruiken in plaats van een if-else-blok, u deze nog steeds in een beknopte one-liner kunt krijgen met behulp van een ternaire operator, waardoor u het beste krijgt van beide werelden.
|
Het tweede codeblok (of het alternatief van Mark Baker, dat precies hetzelfde zal werken) is beter. Ik ben niet helemaal zeker van PHP, maar in veel andere programmeertalen zou het negeren van een variabele vrijwel zeker een fout opleveren. Met het tweede blok initialiseer je de variabele tenminste naar een waarde of geheugenlocatie.
Foutonderdrukking zou vaker moeten worden gebruikt als u verwacht dat een functie een verwachte fout in het eindproduct genereert (dit zal echter meestal niet het geval zijn).
Succes!
Dennis M.
|
Uw antwoord
StackExchange.ifUsing ("editor", function () {
StackExchange.using ("externalEditor", function () {
StackExchange.using ("snippets", function () {
StackExchange.snippets.init ();
});
});
}, "code-snippets");
StackExchange.ready (function () {
var channelOptions = {
tags: "" .split (""),
id: "1"
};
initTagRenderer ("". split (""), "" .split (""), channelOptions);
StackExchange.using ("externalEditor", function () {
// Moet de editor na fragmenten activeren, als fragmenten zijn ingeschakeld
if (StackExchange.settings.snippets.snippetsEnabled) {
StackExchange.using ("snippets", function () {
createEditor ();
});
}
anders {
createEditor ();
}
});
functie createEditor () {
StackExchange.prepareEditor ({
useStacksEditor: false,
heartbeatType: 'antwoord',
autoActivateHeartbeat: false,
convertImagesToLinks: waar,
noModals: waar,
showLowRepImageUploadWarning: true,
ReputationToPostImages: 10,
bindNavPrevention: true,
postfix: "",
imageUploader: {
brandingHtml: "Aangedreven door \ u003ca href = \" https: //imgur.com/ \ "\ u003e \ u003csvg class = \" svg-icon \ "width = \" 50 \ "hoogte = \" 18 \ "viewBox = \ "0 0 50 18 \" fill = \ "none \" xmlns = \ "http: //www.w3.org/2000/svg \" \ u003e \ u003cpath d = \ "M46.1709 9.17788C46.1709 8.26454 46.2665 7.94324 47.1084 7.58816C47.4091 7.46349 47.7169 7.36433 48.0099 7.26993C48.9099 6.97997 49.672 6.73443 49.672 5.93063C49.672 5.22043 48.9832 4.61182 48.1414 4.61182C47.4335 4.61182 46.7256 4.91628 46.0943 46.7256 4.91628 46.0845 43.1481 6.59048V11.9512C43.1481 13.2535 43.6264 13.8962 44.6595 13.8962C45.6924 13.8962 46.1709 13.2535 46.1709 11.9512V9.17788Z \ "/ \ u003e \ u003cpath d = \" M32.492 10.1419C32.492 14.045.14.07 37.492 14.045.14.07 37.045 14.6954 41.5985 12.6954 41.5985 10.1419V6.59049C41.5985 5.28821 41.1394 4.66232 40.1061 4.66232C39.0732 4.66232 38.5948 5.28821 38.5948 6.59049V9.60062C38.5948 10.8521 38.2696 11.520455 37.0451 11.5954.545.85.8 521 35.4954 9.60062V6.59049C35.4954 5.28821 35.0173 4.66232 34.0034 4.66232C32.9703 4.66232 32.492 5.28821 32.492 6.59049V10.1419Z \ "/ \ u003e \ u003cpath fill-rule = \" evenodd \ "clip-rule = \" evenodd \ "d = \ "M25.6622 17.6335C27.8049 17.6335 29.3739 16.9402 30.2537 15.6379C30.8468 14.7755 30.9615 13.5579 30.9615 11.9512V6.59049C30.9615 5.28821 30.4833 4.66231 29.4502 4.66231C28.9913 4.63.507 28.455 .1369 4.56087 21.01346,57349 21,0134 9.27932C21.0134 11,9852 23,003 13,913 25,3754 13.913C26.5612 13,913 27,4607 13,4902 28,1109 12.6616C28.1109 12,7229 28,1161 12,7799 28,121 12.8346C28.1256 12,8854 28,1301 12,9342 28,1301 12.983C28.1301 14,4373 27,2502 15,2321 25,777 15.2321C24.8349 15,2321 24,1352 14,9821 23,5661 14.7787C23.176 14.6393 22.8472 14.5218 22.5437 14.5218C21.7977 14.5218 21.2429 15.0123 21.2429 15.6887C21.2429 16.7375 22.9072 17.6335 25.6622 17.6335ZM24.1317 9.21932C24.1317 7 ... 11.5116 26.1024 11.5116C24.9737 11.5116 24.1317 10.6491 24.1317 9.27932Z \ "/ \ u003e \ u003cpath d = \" M16.8045 11.9512C16.8045 13.2535 17.2637 13.8962 18.2965 13.8962C19.3298 13.8962C19.3298 13.8912.46 4.62866 16.4027 4.62866C15.1594 4.62866 14.279 4.98375 13.3609 5.88013C12.653 5.05154 11.6581 4.62866 10.3573 4.62866C9.34336 4.62866 8.57809 4.89931 7.9466 5.5079C7.58314 4. 9328 7.10506 4.66232 6.51203 4.66232C5.47873 4.66232 5.00066 5.28821 5.00066 6.59049V11.9512C5.00066 13.2535 5.47873 13.8962 6.51203 13.8962C7.54479 13.8962 8.0232 13.2535 8.0232 11.9512V8.90741C3 ... .9512C10.893 13.2535 11.3711 13.8962 12.4044 13.8962C13.4375 13.8962 13.9157 13.2535 13.9157 11.9512V8.90741C13.9157 7.58817 14.3365 6.91179 15.4269 6.91179C16.4027 6.91179 16.8045 7.58817 16.8045 M3 8.94108 \ u003c "/" .31675 6.59049C3.31675 5.28821 2.83866 4.66232 1.82471 4.66232C0.791758 4.66232 0.313354 5.28821 0.313354 6.59049V11.9512C0.313354 13.2535 0.791758 13.8962 1.82471 13.8962C2.85798 13.8962 3.31675 13.85 ... "M1.87209 0.400291C0.843612 0.400291 0 1.1159 0 1.98861C0 2.87869 0.822846 3.57676 1.87209 3.57676C2.90056 3.57676 3.7234 2.87869 3.7234 1.98861C3.7234 1.1159 2.90056 0.400291 1.87209 0.400E \" Z # 1 \ u003c / svg \ u003e \ u003c / a \ u003e ",
contentPolicyHtml: "Gebruikersbijdragen gelicentieerd onder \ u003ca href = \" https: //stackoverflow.com/help/licensing \ "\ u003ecc by-sa \ u003c / a \ u003e \ u003ca href = \" https://stackoverflow.com / legal / content-policy \ "\ u003e (contentbeleid) \ u003c / a \ u003e",
allowUrls: waar
},
onDemand: waar,
discardSelector: ".discard-answer"
, onmiddellijkShowMarkdownHelp: true, enableTables: true, enableSnippets: true
});
}
});
Bedankt voor het bijdragen aan een antwoord op Stack Overflow!
Zorg ervoor dat u de vraag beantwoordt. Geef details en deel uw onderzoek!
Maar vermijd ...
Om hulp, opheldering vragen of reageren op andere antwoorden.
Uitspraken doen op basis van meningen; ondersteun ze met referenties of persoonlijke ervaring.
Bekijk onze tips voor het schrijven van goede antwoorden voor meer informatie.
Concept opgeslagen
Concept verwijderd
Meld u aan of log in
StackExchange.ready (function () {
StackExchange.helpers.onClickDraftSave ('# login-link');
});
Meld u aan met Google
Meld u aan met Facebook
Meld u aan met e-mail en wachtwoord
Verzenden
Post als gast
Naam
E-mail
Vereist, maar nooit getoond
StackExchange.ready (
functie () {
StackExchange.openid.initPostLogin ('. New-post-login', 'https% 3a% 2f% 2fstackoverflow.com% 2fquestions% 2f4441528% 2fis-arraypossible-missing-key-an-anti-pattern% 23new-answer', ' vraag_pagina ');
}
);
Post als gast
Naam
E-mail
Vereist, maar nooit getoond
Plaats uw antwoord
Gooi weg
Door op "Plaats uw antwoord" te klikken, gaat u akkoord met onze servicevoorwaarden, privacybeleid en cookiebeleid
Niet het antwoord waar je naar zoekt? Blader door andere vragen met de tag php arrays operators foutonderdrukking of stel uw eigen vraag.